Skip to content

[SYCL][E2E] Re-enable KernelAndProgram/free_function_kernels.cpp e2e test #20936

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
KornevNikita merged 6 commits into from
Jan 27, 2026
Merged

[SYCL][E2E] Re-enable KernelAndProgram/free_function_kernels.cpp e2e test #20936

KornevNikita merged 6 commits into from
Jan 27, 2026
+1 −2

Conversation

@dyniols
Copy link
Contributor

@dyniols dyniols commented Dec 18, 2025

This PR re-enables KernelAndProgram/free_function_kernels.cpp e2e test for Intel GPU's based on #20244 this issue.

It should be fixed in igc_2.25.x based on this ticket GSD-11922

@dyniols
Copy link
Contributor Author

dyniols commented Jan 23, 2026

Since IGC is updated to v2.27.10 the patch on their side fixing issue regarding this test should be there too. I will re-enable it too.
@sarnex

@sarnex
Copy link
Contributor

sarnex commented Jan 23, 2026 •
edited
Loading

Sure. The driver on the Windows machines is unchanged though and needs to be manually updated, and I have no idea what the IGC version in the latest Win driver is but probably it's older than v2.27.10, feel free to try it tho

@dyniols
Copy link
Contributor Author

dyniols commented Jan 23, 2026

Sure. The driver on the Windows machines is unchanged though and needs to be manually updated, and I have no idea what the IGC version in the latest Win driver is but probably it's older than v2.27.10, feel free to try it tho

Ok I will keep this in mind. Thanks.

@dyniols
Copy link
Contributor Author

dyniols commented Jan 26, 2026

@sarnex Ok you were right. Looks like IGC version in the latest Win driver is older than v2.27.10 so that is why the failures in windows CI.

I can only e-enable it for Linux or shall we wait for the windows driver upgrade?

@sarnex
Copy link
Contributor

sarnex commented Jan 26, 2026

I would just re-enable it for Linux now, the Windows driver is really slow, it could be like 6 months or more before we see the version we need

@dyniols
Copy link
Contributor Author

dyniols commented Jan 26, 2026

I would just re-enable it for Linux now, the Windows driver is really slow, it could be like 6 months or more before we see the version we need

Ok I re-enable for linux only

@dyniols dyniols marked this pull request as ready for review January 26, 2026 16:58
@dyniols dyniols requested a review from a team as a code owner January 26, 2026 16:58
@dyniols dyniols requested a review from uditagarwal97 January 26, 2026 16:58
uditagarwal97
uditagarwal97 reviewed Jan 26, 2026
View reviewed changes
@dyniols
Copy link
Contributor Author

dyniols commented Jan 27, 2026

@intel/llvm-gatekeepers Can we merge this PR ? Thanks

@KornevNikita KornevNikita merged commit b1d48bd into intel:sycl Jan 27, 2026
43 of 46 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@uditagarwal97 uditagarwal97 uditagarwal97 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@dyniols @sarnex @uditagarwal97 @KornevNikita